[PATCH] D23667: [AAP] (3) Add AAP backend stub

Joerg Sonnenberger via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 4 09:04:24 PST 2016


joerg added a comment.

Just being a bit pedantic here.



================
Comment at: lib/Target/AAP/CMakeLists.txt:5
+  AAPTargetMachine.cpp
+  )
+
----------------
Extra spaces.


================
Comment at: lib/Target/AAP/LLVMBuild.txt:1
+;===- ./lib/Target/AAP/LLVMBuild.txt -------------------------*- Conf -*--===;
+;
----------------
Missing - for line length.


================
Comment at: lib/Target/AAP/TargetInfo/AAPTargetInfo.cpp:27
+// Placeholder. This will be removed when the MC support has been
+// introduced
+extern "C" void LLVMInitializeAAPTargetMC() {
----------------
Missing period.


================
Comment at: lib/Target/AAP/TargetInfo/CMakeLists.txt:3
+  AAPTargetInfo.cpp
+  )
----------------
Extra space.


https://reviews.llvm.org/D23667





More information about the llvm-commits mailing list