[PATCH] D37256: [Nios2] adding subtarget, basic infrastructure for frame, instructions and registers

Andrei via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 21 01:04:35 PDT 2017


AndreiGrischenko added a comment.

Thanks Eric for your help.



================
Comment at: llvm/trunk/lib/Target/Nios2/Nios2TargetMachine.h:22
 class Nios2TargetMachine : public LLVMTargetMachine {
+  Nios2Subtarget DefaultSubtarget;
+
----------------
echristo wrote:
> Please don't have a default subtarget. There's no reason to do so and it just confuses things. 
OK, thanks for the comment. I planned to address it in the next patch.


Repository:
  rL LLVM

https://reviews.llvm.org/D37256





More information about the llvm-commits mailing list