gulfem added inline comments.

================
Comment at: llvm/utils/gn/secondary/llvm/lib/Transforms/Utils/BUILD.gn:64
     "PromoteMemoryToRegister.cpp",
+    "RelLookupTableConverter.cpp"
     "SSAUpdater.cpp",
----------------
thakis wrote:
> leonardchan wrote:
> > Good that you added this, but I think Nico has a bot that automatically 
> > updates these BUILD.gn files so manually updating them may not be necessary.
> Please don't touch gn files unless you use them. Simple file additions in 
> cmake files are synced automatically http://github.com/llvmgnsyncbot
> 
> You forgot to add the trailing comma and now I had to fix it up manually 
> instead of doing nothing :P
Sorry about that Niko. I can fix it, so you don't need to do anything.
Leo actually pointed that out, but I thought that manually changing it won't do 
any harm.
Apparently it did though!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94355/new/

https://reviews.llvm.org/D94355

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to