Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Headers-Klasse #70

Open
LukasSiefke opened this issue Dec 7, 2019 · 5 comments · May be fixed by #147
Open

Headers-Klasse #70

LukasSiefke opened this issue Dec 7, 2019 · 5 comments · May be fixed by #147
Labels
good first issue Good for newcomers

Comments

@LukasSiefke
Copy link
Collaborator

nvrtc scheint ja auch ohne explizite Headers-Angabe die headers zu finden zumindest bei headern im selben Ordner...
Vielleicht Erzeugung mit Headern mit Namen != Pfadnamen zulassen?

@LukasSiefke LukasSiefke self-assigned this Dec 7, 2019
@zeratax
Copy link
Owner

zeratax commented Dec 8, 2019

und damit das ganze mehr Sinn macht und #72 wahrscheinlich nicht wichtig ist, sollte Headers vielleicht bei compilezeit schon die Header einlesen? Source genauso maybe? dann müsste man nicht immer alles rumkopieren...

@LukasSiefke
Copy link
Collaborator Author

Source bekommt doch sowieso schon zur Laufzeit einen String
Ich denke nicht, dass mal alle Headers schon zur Komplilezeit einlesen kann, da später, wenn das dann von Scala aus benutzt wird und dort Header included werden sollen, will man ja dafür nicht die Bibliothek neu kompilieren
Aber das rumkopieren finde momentan auch nicht so optimal

@zeratax
Copy link
Owner

zeratax commented Dec 8, 2019

ah ja garnicht an scala gedacht.

aus der nvrtc doc klingt es halt so als würde nvrtc sachen schon finden, die header beim kompilieren anzugeben ist nur so dafür da, falls man nicht sicher sein kann, dass sie da sein werden

Alternatively, the compile option -I can be used if the header is guaranteed to exist in the file system at runtime.

obwohl wir ja auch nicht -I benutzt haben...

@zeratax zeratax added the good first issue Good for newcomers label Dec 14, 2019
@LukasSiefke LukasSiefke removed their assignment Dec 15, 2019
@zeratax
Copy link
Owner

zeratax commented Jan 14, 2020

wollte nochmal sagen, dass ich dafür bin headers so zu ändern, dass die etwa so aufgerufen werden

Header header1{"typedef struct {\n
  int x;\n
} header1;", "header1.h"}

Source function{"#include \"header1.h\"\n
extern \"C\" __global__ header1 function() {\n
  header1 a;\n
  a.x = 5;\n
  return a;\n", Headers{header1}};

und nichts von der festplatte geladen wird, da das nvrtc ja schon selbst macht und uns nur die tests kompliziert

@LukasSiefke
Copy link
Collaborator Author

Das finde ich sehr sinnvoll

@zeratax zeratax linked a pull request Mar 28, 2020 that will close this issue
@zeratax zeratax added this to the v1 basic executor milestone Apr 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants